Raklion: fall and push of Selupan, skill multipliers, and the closed hatchery gate - #994

Follow-up of #977.

Summary

@ze-dom confirmed in #977 that Selupan hits players when it falls from the sky at the start of the battle (video). Until now, the fall was only the animation of its appearance: it dealt no damage and pushed nobody.

This PR adds:

• the damage and the push of the fall;
• the push of the frost shock;
• the per-class limit of the push.

It also fixes two things:

• the gate to the hatchery let players enter while the hatchery was closed;
• the damage multipliers of the Selupan skills from Add the Raklion event (Battle of Selupan) #977 threw an exception on the first use of each skill, and were missing afterwards.

Behaviour of the original game

The known server sources (IGCN, MuEmu, kessiler) agree on this behaviour:

• The fall (FirstSkill) is used exactly once, at the first step of the battle. It uses skill 253 with the monster skill unit 37 ("Selupan - Fall"). The unit hits every player closer to Selupan than 4 fields (ScopeType 0, ScopeValue 4), measured from Selupan, not from its target. The damage multiplier is 2.5, as @ze-dom pointed out.
• The push (skill element type 9, gObjBackSpring2) moves each hit player away from Selupan, field by field. The fall pushes by 4 fields (element 51), the frost shock by 10 (element 50). The push stops at fields which aren't walkable, including the safezone. Stunned and frozen players aren't pushed.
• Only dark knights, magic gladiators and dark lords are pushed by the full distance. The other classes are pushed by 2 fields at most. All three sources have this in gObjBackSpring2.
• The hatchery gate doesn't move the player while the hatchery is closed. The player stays where it is and gets the message "The Hatchery Gate is closed, you cannot enter."

Two details are intentionally left out:

• In the sources, the damage of the fall goes to the main target once per player in range, while the push goes to each player in range. That looks like a bug. Here, each player in range is hit once.
• The original push bounces back from a blocked field. Here, it just stops, like the existing push of Earthshake.

Changes

• SelupanIntelligence:
  • When Selupan appears, it hits all valid targets closer to it than FallRadius. It still shows the fall animation as before. The damage uses the new skill Selupan Fall (253), so its multiplier applies, like the other attack skills of Selupan.
  • The fall and the ice strike (frost shock) push the hit players away (PushAwayAsync). The push works like the one of Earthshake: it uses Player.MoveAsync, so the clients are updated.
• RaklionEventDefinition, new settings:
  • FallRadius (4);
  • FallPushDistance (4);
  • IceStrikePushDistance (10);
  • LimitedPushDistance (2);
  • FullyPushedCharacterClassNumbers (the dark knight, magic gladiator and dark lord classes);
  • GetPushDistance(...), which applies the per-class limit.
• WarpGateAction: before the warp, it calls the new plugin point IWarpGateEnteringPlugIn, which can deny the entrance by cancelling it. RaklionPlugIn implements it: RaklionContext.CanEnterThroughGateAsync denies the hatchery while it's closed and shows the message. So WarpGateAction doesn't know about any event, and other events with a closed map can use the same plugin point. Like the original game (MuEmu's ERROR_JUMP), a denied player is warped to its current position. A "map change failed" isn't enough, because the game client keeps showing its loading screen after it entered a gate until a map change is completed. Before, the player entered the closed hatchery, got the message and was moved to the start of Raklion by the next tick.
• MonsterAttributeHolder.GetOrCreateAttribute: this threw a NotImplementedException. When Selupan attacked with a skill, EnsureSkillAttributes failed while adding the relationship of the damage multiplier, because the relationship needs the SkillMultiplier of the monster. The skill attributes were already assigned to the SkillEntry at that point, so every later attack used them without the multiplier. The monster now returns its composed attribute if elements were added to it, and a constant element of its stat value otherwise.
• Data:
  • SkillNumber.SelupanFall = 253;
  • the skill with its multiplier (2.5, InputOperator.Maximum like the other three) for new databases;
  • AddSelupanFallSkillUpdatePlugIn for existing ones. It reuses AddRaklionEventUpdatePlugIn.CreateSkill, and applying it twice changes nothing.
• Static code analysis: addresses the remaining Codacy findings of the Selupan skills from Add the Raklion event (Battle of Selupan) #977. This was committed after the merge.

Update version: this uses UpdateVersion 120, because 119 is used by the Imperial Guardian event in #992. If this is merged first, I'll renumber it.

Tests

• SelupanIntelligenceTest:
  • the damage multiplier of a skill is calculated with the attributes of the monster (this failed with the NotImplementedException before);
  • only the players closer than the fall radius are hit; a player exactly at the radius or further away isn't;
  • a hit player is pushed away by the push distance;
  • the push stops in front of a field which isn't walkable.
• RaklionEventDefinitionTest: the push distance of each character class.
• RaklionHatcheryGateTest: the closed hatchery can't be entered through a gate, other maps can, and the plugin denies nothing while the event isn't running.
• RaklionDataTest now also covers the fall skill and its multiplier, for a new database and for the updates applied twice.

Built with -p:ci=true; all four test suites pass. Tested in game: the fall hits and pushes the players around Selupan. The exception of the multiplier was found in the server log of that test.

🤖 Generated with Claude Code